-
Notifications
You must be signed in to change notification settings - Fork 132
Fix incorrect image upload error message #11493
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…o-trunk Merge 18.6 code freeze to trunk
# Conflicts: # WooCommerce/src/main/kotlin/com/woocommerce/android/util/FeatureFlag.kt
…flag-m2 [Dynamic dashboard] feature flag for m2
…eader [Dynamic Dashboard] Introduce a component to re-use as header in the filterable cards
Attempt to fix crash on ProductListToolbarHelper
📲 You can test the changes from this Pull Request in WooCommerce Android by scanning the QR code below to install the corresponding build.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for taking care of this @0nko, it works well.
Before merging this, it seems to me this is a serious bug, shouldn't we include it in the current beta version?
…nt-empty-pos-activity [Woo Pos] FF, entry point and empty pos activity
Merge 18.6-rc-2 into trunk
f8258be
to
900d37e
Compare
Thanks @hichamboushaba. I've rebased the branch onto |
Thanks @0nko, there is a failing unit test that's preventing the merge. |
This PR fixes the wrong private site status message for self-hosted sites, which prevents the app from uploading images.
To test: